Skip to content

fix: should update initial chunks correctly with filename#1132

Merged
alexander-akait merged 1 commit intowebpack:masterfrom
JSerFeng:fix/hmr-for-initial-chunk
Jul 31, 2025
Merged

fix: should update initial chunks correctly with filename#1132
alexander-akait merged 1 commit intowebpack:masterfrom
JSerFeng:fix/hmr-for-initial-chunk

Conversation

@JSerFeng
Copy link
Copy Markdown
Contributor

@JSerFeng JSerFeng commented Jul 18, 2025

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Fix #1133

Initial css chunk cannot update if user specify filename or chunkFilename. Because MiniCssExtractPlugin has GetChunkFilenameRuntimeModule and force the allChunks parameter to be false, that will cause that runtime cannot get correct url for updated initial css chunk.

This change aligns behavior of webpack's native css support.

Breaking Changes

Additional Info

@JSerFeng
Copy link
Copy Markdown
Contributor Author

How can I add test for this case, seems HMR.test is not what I'm looking for

@JSerFeng JSerFeng changed the title fix: should contains GetFilenameRuntime for initial chunks fix: should update initial chunks correctly with filename Jul 18, 2025
@alexander-akait
Copy link
Copy Markdown
Member

@JSerFeng Sorry for delay, can you provide a steps to reproduce? I tried your repo and everything works...

@alexander-akait
Copy link
Copy Markdown
Member

I see, yeah, it is a bug, let's merge, I will fix tests, thanks

@alexander-akait alexander-akait merged commit dab023f into webpack:master Jul 31, 2025
3 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Initial style cannot update

2 participants